Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fixes and drastic improvement in frame move smoothness #1864

Merged
merged 2 commits into from
Jun 18, 2024
Merged

Conversation

MrTJP
Copy link
Owner

@MrTJP MrTJP commented Jun 2, 2024

  • Fix case of connectable block entities re-connecting to blocks in process of being removed during their neighbor update call. Same issue was fixed for parts in af30355
  • Fix issue with blocks being moved by frame motor where the server-side blocks may try to update their client counterparts before they are actually moved on the client.
    • In previous logic, the server completes the entire move before client. The last step of the movement is to do neighbor updates, where block entities may update their client counterparts of their new state via packet.
    • Now, movement is staggered across server and client in this order:
      • Server silently moves blocks and block entities without neighbor updates
      • Client completes full move
      • Server finishes move by performing block updates, etc
    • As a byproduct, since nothing re-renders chunk until movement is fully complete on both sides, the rendering is extremely smooth in semi-blocking and blocking chunk modes :) (Threaded mode is smoother than before but still presents artifacts).

@MrTJP MrTJP changed the title Bug fixes Bug fixes and drastic improvement in frame move smoothness Jun 2, 2024
@MrTJP MrTJP merged commit 87aa5d8 into main Jun 18, 2024
2 checks passed
@MrTJP MrTJP deleted the patch-06022024 branch June 18, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant